hardware: remove legacy model validation classification - #54
Merged
mksolutionsuniversal merged 2 commits intoSep 8, 2026
Merged
Conversation
mksolutionsuniversal
left a comment
mksolutionsuniversal
left a comment
Owner
Author
There was a problem hiding this comment.
Maintainer audit PASS.
- Final head:
2a03bd6926dce4589c846725d4fa84610cbea236 - 2 commits, 39 changed files.
SupportedIntelMacnow derives from Apple hardware + Intel CPU without a model registry/whitelist.- Legacy
ModelValidationLevel/ model-support registry types are removed. - Supported UI no longer shows tested/untested or model-validation status; unsupported/incomplete platforms retain useful details.
- Compatibility/diagnostic reports retain model identity, platform support, Gaming eligibility, power/Restore state and live fan capability diagnostics.
VerifiedHardwareModels.MacBookPro16_1remains only for justified legacy schema-v1 downgrade compatibility and the bounded research round-trip tool; it is not a production writer permission gate.- No production FanControl, PowerManagement, Profiles, SettingsBackup or driver write semantics changed.
- Local validation: 632/632 PASS, 0 warnings / 0 errors, diff-check PASS, packaging/safety scan PASS, zero physical SMC writes.
- PR CI #180 on the exact head: full
Build and testSUCCESS including Gate 5D-B and release packaging validation.
Ready for squash merge.
mksolutionsuniversal
deleted the
release/0.5.0-rc.2-remove-model-validation
branch
September 8, 2026 19:35
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Remove the obsolete per-model validation-classification layer from the current BCPC product path.
SupportedIntelMacis now determined directly from detected Apple hardware plus an Intel CPU. Exact Mac model identity remains hardware/reporting context and may still be used where there is a specific compatibility or research reason, but it does not grant normal processor-profile or fan-write permission.Product behavior
SupportedIntelMacwithout a model whitelist.95% / 95%+ Boost Disabled processor policy forSupportedIntelMac.Model validationrow.Platform supportin Compatibility;Detailsis shown only for unsupported or incomplete detection states.Cleanup
Removed obsolete production types:
ModelValidationLevelModelSupportRegistryIModelSupportRegistryModelSupportDefinitionVerifiedHardwareModels.MacBookPro14_3was also removed as an obsolete fixture constant.VerifiedHardwareModels.MacBookPro16_1intentionally remains only for bounded legacy/research uses, including the 0.4.0-compatible legacy ownership-marker schema and the dedicated physical SMC round-trip research tool. These uses do not grant application fan-write permission.Safety boundary
No production changes were made under:
FanControl/PowerManagement/Profiles/SettingsBackup/drivers/No SMC key, payload, mask, RPM conversion, writer ordering, ownership journal, crash recovery, Restore ordering, or processor target was changed.
Validation
Local validation on the branch:
git diff --check: PASSRelease note
This is a pre-release architecture/UI cleanup for the
0.5.0-rc.2development target. It does not create or publish the release; final release-prep documentation and artifact checkpoint remain separate.